Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(catalog-generator): Add tests for the Catalog generator #1519

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

lordrip
Copy link
Member

@lordrip lordrip commented Oct 2, 2024

Context

At this time, we're adding tests for the ConfigBean class and the GenerateCommandTest class.

Note

This PR doesn't fulfill the entire issue but is another step towards adding more coverage to the catalog generator.

fix: #1250

@lordrip lordrip requested review from apupier and igarashitm October 2, 2024 12:52
@lordrip lordrip force-pushed the chore/add-tests-catalog-generator branch from bb0b444 to 66882b8 Compare October 2, 2024 13:01
At this time, we're adding tests for the `ConfigBean` class and the
`GenerateCommandTest` class.

fix: KaotoIO#1250
@lordrip lordrip force-pushed the chore/add-tests-catalog-generator branch from 66882b8 to ccc4435 Compare October 2, 2024 13:07
Copy link

sonarcloud bot commented Oct 2, 2024

Copy link

codecov bot commented Oct 2, 2024

Codecov Report

Attention: Patch coverage is 2.31481% with 211 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@458400e). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...kaoto/camelcatalog/generator/CatalogGenerator.java 1.91% 205 Missing ⚠️
...amelcatalog/generator/CatalogGeneratorBuilder.java 0.00% 6 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1519   +/-   ##
=======================================
  Coverage        ?   69.97%           
  Complexity      ?       62           
=======================================
  Files           ?      276           
  Lines           ?     7726           
  Branches        ?     1514           
=======================================
  Hits            ?     5406           
  Misses          ?     2270           
  Partials        ?       50           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lordrip lordrip merged commit 3907213 into KaotoIO:main Oct 2, 2024
12 checks passed
@lordrip lordrip deleted the chore/add-tests-catalog-generator branch October 2, 2024 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Write the catalog files at the end of the process
2 participants